Skip to content

Conversation

@Camille0907
Copy link

@Camille0907 Camille0907 commented Oct 10, 2025

Add a new module to manage international credit transfers.
This adds the possibility to add an Intermediary bank on a res.partner.bank. When generating the pain file for an International Credit Transfer payment method, the intermediary bank account info will be added in CstmrCdtTrfInitn > PmtInf > CdtTrfTxInf > IntrmyAgt1 > FinInstnId
For now the module only supports pain.001.001.03

A lot of the code is very close from the module account_banking_sepa_credit_transfer: since a lot of copy/paste have been done, we've kept the Copyright of the original module

Also rename finalize_sepa_file_creation into finalize_pain_file_creation. Indeed, this method can also be used for non-sepa pain files

…e_creation

Indeed, this method can also be used for non-sepa pain files
@Camille0907 Camille0907 force-pushed the 17.0-add-account_banking_international_credit_transfer branch 3 times, most recently from e3a3f47 to e743a05 Compare October 10, 2025 14:23
Add a new module to manage international credit transfers.
This adds the possibility to add an Intermediary bank on a `res.partner.bank`. When generating the pain file for an International Credit Tranfer payment method, the intermediary bank account info will be added in CstmrCdtTrfInitn > PmtInf > CdtTrfTxInf > IntrmyAgt1 > FinInstnId
For now the module only supports pain.001.001.03

A lot of the code is very close from the module ``account_banking_sepa_credit_transfer``: since a lot of copy/paste have been done, we've kept the Copyright of the original module
@Camille0907 Camille0907 force-pushed the 17.0-add-account_banking_international_credit_transfer branch from e743a05 to 648d708 Compare October 10, 2025 14:50
Copy link

@yankinmax yankinmax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants